fix: ls -R hierarchy + tracking baseline + incompatible flags (#714, #561)#799
Open
pszymkowiak wants to merge 1 commit intodevelopfrom
Open
fix: ls -R hierarchy + tracking baseline + incompatible flags (#714, #561)#799pszymkowiak wants to merge 1 commit intodevelopfrom
pszymkowiak wants to merge 1 commit intodevelopfrom
Conversation
…561) - ls -R: preserve directory hierarchy with indentation (files under their dir) - Tracking: compare against plain ls (not ls -la) for honest savings - Passthrough for flags incompatible with -l parser: -1, -d, -F, -i, -p Tested with 3 LLMs (llama, phi4, claude) on 20 ls variants: 100% comprehension. Signed-off-by: Patrick szymkowiak <patrick.szymkowiak@innovtech.eu>
pszymkowiak
added
bug
Collaborator
Author
📊 Automated PR Analysis
SummaryFixes recursive ls output to display directory hierarchy with indentation instead of flat listing, changes the tracking baseline to compare against plain ls rather than ls -la, and adds passthrough for flags incompatible with the -l parser (-1, -d, -F, -C, -m, -x, -i, -p). Review Checklist
Analyzed automatically by wshm · This is an automated analysis, not a human review. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #714, Closes #561
Changes
rtk ls -Rflattens recursive output and loses hierarchy context #714): recursive output now shows directory structure with indentationlsnotls -la#561): savings compared against plainls(notls -la)-1,-d,-F,-i,-ppassthrough to raw lsBefore/After
LLM comprehension test
20 ls variants tested with llama3.1:8b, phi4:14b, Claude — 100% comprehension on hierarchy.